Skip to content

Gashaponsition#2044

Open
MothHours wants to merge 5 commits into
HardLightSector:masterfrom
MothHours:Gashaponsition
Open

Gashaponsition#2044
MothHours wants to merge 5 commits into
HardLightSector:masterfrom
MothHours:Gashaponsition

Conversation

@MothHours

Copy link
Copy Markdown
Contributor

About the PR

Copied/reworked the Egglaying trait code to need a new reagent PrizeballJuice, Localized as: "Juice that makes you prizeball" that causes a player with the trait to form and be able to lay prizeballs.

Why / Balance

Just another fun flavor/kink trait, I do not think it should impact game balance as the number of tickets needed per prizeball is the same as the prize counter.

Technical details

Copies most of the LewdEggLaying code into equivalently named files for the trait.

How to test

Give your test character the trait
Consume prize tickets
Wait.
Lay Prizeball action should pop up, clicking it should lay a prizeball.

Add an entity with the "actor" server and client component.
"Lay Prizeball inside" should appear in that entities verb menu.

Media

Breaking changes

Changelog

🆑 add: Added Gashaponsition, a new trait that lets you lay prizeballs.

Updated Prizeball laying systems to match the equivalent LewdEggLaying System, and made the trait appear in the traits tab

@AutumnalModding AutumnalModding left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good I think. Got a few comments, mostly nitpicks though.

@@ -1,4 +1,5 @@
flavor-woof = like woof
flavor-highscore = Like a sweet sweet highscore.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be lowercase.

using Robust.Shared.Prototypes;
using Content.Shared.Animals.Systems;

namespace Content.Shared.Animals.Components; // HL: Moved this to Shared so the client can use it for verb drawing.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for HL comments in HL namespace


namespace Content.Client.Animals.Systems;

// HL: This is just an empty class so that the matching code in SharedPrizeballLayingSystem can run on the client.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for HL comments in HL namespace

@@ -0,0 +1,26 @@
using Content.Shared.EntityEffects;
using Content.Shared.Animals.Components; // HL: Moved the LewdEggLayingComponent to Shared

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for HL comments in HL namespace

Comment on lines +46 to +60
- type: Food
solution: food
delay: 0.25
forceFeedDelay: 1
- type: FlavorProfile
flavors:
- paper
- type: BadFood
- type: SolutionContainerManager
solutions:
food:
maxVol: 1
reagents:
- ReagentId: PrizeballJuice
Quantity: .1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing HL comments

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be in HL namespace

@AutumnalModding

Copy link
Copy Markdown
Collaborator

@MothHours why re-request a review without actually pushing changes...?

@AutumnalModding

Copy link
Copy Markdown
Collaborator

Also your linter is failing:


Error: /Prototypes/_HL/Reagents/fun.yml(40,11)  No Content.Shared.Nutrition.FlavorPrototype found with id highscore
Error: /Prototypes/_HL/Traits/lewd.yml(69,7)  Field "maxEggs" not found in "Content.Shared.Animals.Components.PrizeballLayingComponent".

@MothHours

Copy link
Copy Markdown
Contributor Author

@MothHours why re-request a review without actually pushing changes...?

Sorry! I did not mean to do that, very new to Github.

Removed the HL tag from comments on code in HL folder
Fixed the flavor
Fixed maxPballs being maxEggs in lewd.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants